-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added e2e test for operator resolution #103
Conversation
test/e2e/install_test.go
Outdated
Name: fmt.Sprintf("operator-%s", rand.String(8)), | ||
}, | ||
Spec: operatorv1alpha1.OperatorSpec{ | ||
PackageName: "prometheus", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit, use a constant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated 😃
@@ -0,0 +1,61 @@ | |||
package e2e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
package e2e_test
??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think package e2e is fine - there's only tests and test tooling here...no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that's right, I feel like I usually just see that convention when we have actualpkg
and then when we test it actualpkg_test
/lgtm |
a3acd70
to
385a889
Compare
/lgtm |
…ning e2e tests Signed-off-by: dtfranz <dfranz@redhat.com>
385a889
to
4b51ca6
Compare
New changes are detected. LGTM label has been removed. |
Resolver e2e test for Operator Controller. Creates an Operator Resource specifying prometheus as the packageName then expects resolution to succeed and provide a bundle path.